-
Notifications
You must be signed in to change notification settings - Fork 6
feat: Add activity logging for pay code registrations #75
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Add optional webhook support for monitoring pay code activity - Privacy-preserving: logs only type (free/paid) and domain - Non-blocking notifications (fire-and-forget) - Configure via ACTIVITY_WEBHOOK_URL environment variable
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@CodeRabbit review |
✅ Actions performedReview triggered.
|
📝 WalkthroughWalkthroughThis pull request introduces activity logging for pay code registrations. A new notifications utility sends events (free or paid) to an optional webhook when pay codes are created or redeemed. Configuration and integration points are added to enable this non-blocking logging functionality. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 🧹 Recent nitpick comments
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Adds optional webhook support for logging pay code registration activity.
Changes
src/lib/notifications.tsutility for activity loggingConfiguration
Set
ACTIVITY_WEBHOOK_URLenvironment variable to enable.Testing
Summary by CodeRabbit
Release Notes